feat(phase0): rename /agent-platform/ → /agents/ (IA restructure Phase 0) - #420
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This PR moves the Agents docs from /agent-platform/ to /agents/, updates sidebar/cross-link references, and adds permanent redirects for the old URLs. I found no security issues, and the provided spec context contains no approved repository spec to compare against.
Concerns
- The llms custom set still exposes the old "Agent Platform" naming even though the section URL and sidebar now use Agents.
- The PR includes an unrelated
yamldependency bump and resulting lockfile churn that should be reverted unless it is intentionally required.
Verdict
Found: 0 critical, 0 important, 2 suggestions
Approve with nits
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
a430962 to
7041852
Compare
|
/oz-review |
|
I'm re-reviewing this pull request in response to a review request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This PR performs a broad mechanical rename of the Agents docs URL prefix from /agent-platform/ to /agents/, with sidebar/config updates and matching redirects for moved pages. The migration itself looks consistent, and no security findings or approved spec-context drift were identified.
Concerns
package.jsonremoves the directyamldependency even though build-time code imports it directly inscripts/generate-openapi-json.mjsandsrc/pages/api.astro, making the build rely on a transitive dependency.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
hongyi-chen
left a comment
There was a problem hiding this comment.
approving to unblock, leaving high level thoughts in slack thread
- Move src/content/docs/agent-platform/ → src/content/docs/agents/ - Update all sidebar slug references in src/sidebar.ts - Update astro.config.mjs llms.txt paths - Update 'agents' display-name mapping in CustomSidebar.astro - Fix all /agent-platform/ internal cross-links across 105+ content files - Add 50 permanent (308) redirects in vercel.json Rationale: the Agents tab is labeled 'Agents' but linked to /agent-platform/, creating a confusing pairing with /platform/. The /agents/ vs /platform/ URL distinction is immediately legible. This is standalone technical debt cleanup with no dependency on other IA restructure phases. Co-Authored-By: Oz <oz-agent@warp.dev>
…by sed) The bulk sed pass changed assets/agent-platform/ → assets/agents/ in 15 files, but the assets directory itself was not renamed. This restores the correct paths. Co-Authored-By: Oz <oz-agent@warp.dev>
…aml dep - astro.config.mjs: update llms.txt custom set label/description from 'Agent Platform' → 'Agents' to match the renamed section URL/path - package.json + package-lock.json: remove accidental yaml dep added during local dev (unrelated to this rename, not needed at build time) - agents/capabilities/slash-commands.mdx: fix remaining /agent-platform/ href links that were missed by the initial sed pass Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
The frontmatter title still said "Agent platform FAQs" while the sidebar label already said "Agent FAQs". Aligning them removes the pre-rename copy that was flagged during QA review of the preview. Co-Authored-By: Oz <oz-agent@warp.dev>
5b6a246 to
20bb102
Compare
These two files were added to main after the branch was created and still referenced the old /agent-platform/ prefix; updated all internal links to /agents/. Co-Authored-By: Oz <oz-agent@warp.dev>
…#14649) ## Summary warpdotdev/docs#420 (merged 2026-08-03) renamed `/agent-platform/...` docs URLs as part of the IA restructure Phase 0. 308 redirects keep the old links working today, but the in-app hardcoded `docs.warp.dev/agent-platform/...` links in this client (agent tips, ambient-agent tips, cloud setup guide, AI settings, MCP list page, HOA onboarding, zero-state block, auth secret types, agent SDK ambient CLI output, the bug report issue template, and the bundled `oz-platform` skill) should point at their final canonical paths instead. This is a mechanical link-hygiene rewrite, not a blind `/agent-platform/` -> `/agents/` replace: - Capabilities / local-agents / inference content -> `/agents/...` - Cloud-agents content -> `/platform/...` (not `/agents/cloud-agents/...`), per the docs `vercel.json` `:path*` rewrite plus specific overrides (e.g. `cloud-agents/overview` -> `/platform/`, `cloud-agents/platform` -> `/platform/overview/`) - Special alias: `warp-agents/agent-notifications` -> `agents/capabilities/agent-notifications` - `.github/ISSUE_TEMPLATE/01_bug_report.yml`'s `ai-faqs` link -> `agents/getting-started/faqs/` - Bundled `oz-platform` skill's `computer-use.md` link -> `agents/capabilities/computer-use.md` Out of scope (left untouched per the ticket): `crates/warp_core/src/ui/icons.rs`'s `GeminiEnterpriseAgentPlatform` icon (an SVG asset path, not a docs URL) and `.agents/skills/review-pr-local/SKILL.md` (an internal engineering skill doc, not a shipped in-app/product surface). ### Rework changes -⚠️ [IMPORTANT] `https://docs.warp.dev/platform/environments` returned a 308 redirect to `https://docs.warp.dev/platform/environments/`, leaving a redirect hop. **Implemented**: added the trailing slash at both call sites in `app/src/terminal/view/ambient_agent/tips.rs` (lines 85 and 141), and re-verified with a no-follow `curl` request that it now returns a direct 200. - Additional instruction to re-check every docs URL touched by this PR with a no-follow request: **Implemented**. Re-extracted all 31 unique docs.warp.dev URLs added by this PR's diff and checked each with `curl -sS -o /dev/null -w '%{http_code}'` (no `-L`). The `/platform/environments` URL above was the only one returning a 3xx; every other URL already returned a direct 200. ## Verification - Derived the old -> final path mapping directly from `warpdotdev/docs`' `vercel.json` redirects (including the `/agent-platform/cloud-agents/:path*` -> `/platform/:path*` rule and its specific overrides) rather than assuming a prefix-only replace. - Repo-wide search confirms zero remaining `docs.warp.dev/agent-platform` occurrences in the touched files. - Re-checked every rewritten docs URL touched by this PR against the live site with a **no-follow** `curl -sS -o /dev/null -w '%{http_code}'` (not `-L`): all 31 now resolve directly with HTTP 200, no redirect hop. - `cargo fmt --check` and `cargo check -p warp --all-targets` pass. - Testing-exempt: this is a pure string-literal/copy change (docs href targets only) with no new logic or behavior — no regression test added, per the testing-exempt "pure data/copy" category. Originating thread: https://warpdev.slack.com/archives/C0BDQDW8V5E/p1785772516630019 <!-- factory-agent: {"source":"factory-agent","task_id":"APP-5106","task_source":"linear","task_url":"https://linear.app/warpdotdev/issue/APP-5106/update-in-app-docs-links-from-agent-platform-to-post-docs420-paths","linear_issue_id":"APP-5106","oz_run_id":"019fc861-610b-7584-9d27-0e1c16fdcd60","repo":"warpdotdev/warp","ci_fix_attempts":0,"review_rework_attempts":1} --> --------- Co-authored-by: Oz <oz-agent@warp.dev>
Summary
Renames the Agents documentation URL prefix from
/agent-platform/to/agents/across the docs site. This standalone Phase 0 change removes ambiguity between the Agents section and the Automation Platform section before the broader information architecture restructure.Why now
The navigation label is Agents, but its URL prefix was
/agent-platform/, which was easy to confuse with/platform/. Moving the section now prevents additional pages from accumulating under the legacy prefix and reduces future redirect work.Changes
src/content/docs/agent-platform/tosrc/content/docs/agents/while preserving file history./agents/.308redirects from the legacy/agent-platform/URLs to their/agents/equivalents.assets/agent-platform/; only documentation routes changed.Notes
scripts/generate_rename_redirects.pyis included as the utility used to generate the redirect set.main.Conversation: https://staging.warp.dev/conversation/42fe184c-9601-4da8-b158-200b89b4fd9c
Co-Authored-By: Oz oz-agent@warp.dev